Fix reading columns with dot-separated names from Iceberg #1319
Conversation
mkmkme
added
antalya
bugfix
antalya-25.8
antalya-25.8.14.20001
antalya-25.8.14
antalya-in-upstream
|
@codex review |
|
Codex Review: Didn't find any major issues. You're on a roll. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d6b0a778de
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| /// Strip "current_path." prefix to get child name (preserves dots in child names) | ||
| std::string_view mapped = it->second; | ||
| if (mapped.starts_with(current_path) && mapped.size() > current_path.size() | ||
| && mapped[current_path.size()] == '.') | ||
| return mapped.substr(current_path.size() + 1); |
There was a problem hiding this comment.
When case_insensitive_column_matching is enabled, node.name is rewritten to the query’s casing in processSubtree, so current_path can differ in case from the Iceberg-mapped name. The starts_with check here is case-sensitive; if the user queries a tuple/struct column with different case, the prefix won’t be stripped and child names get built with the full mapped path (e.g., mystruct.MyStruct.child), which then fails tryGetPositionByName and can yield missing tuple elements or errors. Consider normalizing current_path/mapped with the same case-insensitive rules before comparing.
Useful? React with 👍 / 👎.
Fixes #1301. Upstream PR: ClickHouse#94335
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixes an issue when Iceberg columns with dot in names returned NULL as values.
CI/CD Options
Exclude tests:
Regression jobs to run: